Skip to content

Conversation

@eljamm
Copy link
Contributor

@eljamm eljamm commented Jan 9, 2026

The ISO C23, which is the default in GCC 15, changes the meaning of an empty argument list from an unspecified list to no arguments, thus breaking the build.

This is fixed by switching the language standard to gnu89 (dialect of ISO C90).

Tracking: #475479

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot requested a review from wegank January 9, 2026 10:30
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. labels Jan 9, 2026
@eljamm

This comment was marked as outdated.

@eljamm eljamm requested a review from imincik January 9, 2026 12:20
The ISO C23, which is the default in GCC 15, changes the meaning of an
empty argument list from an unspecified list to no arguments, thus
breaking the build.

This is fixed by switching the language standard to `gnu89` (dialect of
ISO C90).
@eljamm eljamm force-pushed the fix/termcap-gcc15 branch from 4e20763 to ea15d85 Compare January 12, 2026 06:47
@Prince213 Prince213 mentioned this pull request Jan 12, 2026
13 tasks
@Prince213
Copy link
Member

Came across this in a last-minute search :P

The package is indeed patchable so I suppose #479265 is better but I'm not very sure about vendoring a patch.

@eljamm
Copy link
Contributor Author

eljamm commented Jan 12, 2026

The package is indeed patchable so I suppose #479265 is better but I'm not very sure about vendoring a patch.

I was patching this too at first, but perhaps changing the language standard is better because it's more resilient to future changes. Some distros like fedora are doing this.

@eljamm
Copy link
Contributor Author

eljamm commented Jan 12, 2026

nixpkgs-review result for #478354

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 478354
Commit: ea15d856214fc700d25de3c6c8ddfae88835071d (subsequent changes)
Merge: 03b790d5f978b95b8fbafce20468738de644a2df

Logs: https://github.com/eljamm/nixpkgs-review-gha/actions/runs/20910625291


x86_64-linux

✅ 4 packages built:
  • gnucap
  • gnucap-full
  • termcap
  • termcap.dev

aarch64-linux

✅ 4 packages built:
  • gnucap
  • gnucap-full
  • termcap
  • termcap.dev

x86_64-darwin (sandbox = relaxed)

✅ 2 packages built:
  • termcap
  • termcap.dev

aarch64-darwin (sandbox = relaxed)

✅ 2 packages built:
  • termcap
  • termcap.dev

@wegank wegank added this pull request to the merge queue Jan 12, 2026
Merged via the queue into NixOS:master with commit 29deff1 Jan 12, 2026
30 of 32 checks passed
@eljamm eljamm deleted the fix/termcap-gcc15 branch January 12, 2026 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants